Skip to content

fix: cache bust http cache on lockfile integrity mismatch#28087

Merged
dsherret merged 2 commits intodenoland:mainfrom
dsherret:fix_cache_bust_lockfile_integrity
Feb 13, 2025
Merged

fix: cache bust http cache on lockfile integrity mismatch#28087
dsherret merged 2 commits intodenoland:mainfrom
dsherret:fix_cache_bust_lockfile_integrity

Conversation

@dsherret
Copy link
Member

@dsherret dsherret commented Feb 12, 2025

Cache busts the http cache when the lockfile integrity doesn't match what's in the cache. This will help when someone's lockfile is in line with the remote server, but their local cache isn't.

@dsherret dsherret requested a review from bartlomieju February 12, 2025 14:32
@dsherret dsherret enabled auto-merge (squash) February 12, 2025 14:52
@@ -1,4 +1,5 @@
Download http://localhost:4545/lockfile/basic/mod.ts
Download http://localhost:4545/lockfile/basic/mod.ts
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It redownloads files if they've never been downloaded. I think it's fine because it's a rare error case.

@dsherret dsherret merged commit a2fafd3 into denoland:main Feb 13, 2025
18 checks passed
@dsherret dsherret deleted the fix_cache_bust_lockfile_integrity branch February 13, 2025 12:23
ry pushed a commit that referenced this pull request Apr 7, 2025
Cache busts the http cache when the lockfile integrity doesn't match
what's in the cache. This will help when someone's lockfile is in line
with the remote server, but their local cache isn't.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants